Skip to content

Conversation

@JanisSaldabols
Copy link
Collaborator

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to make the broker use a dedicated Postgres schema (crosslink_broker) by default, and to ensure the schema exists in the local docker-compose Postgres setup.

Changes:

  • Add a Postgres init script to create the crosslink_broker schema for local docker-compose.
  • Update the broker DB connection string to set search_path=crosslink_broker.
  • Document the schema requirement in the root README.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
broker/init.sql Adds SQL to create the crosslink_broker schema.
broker/docker-compose.yml Mounts init.sql into Postgres init directory for local bootstrap.
broker/dbutil/dbutil.go Changes connection string to force search_path=crosslink_broker.
README.md Notes that the crosslink_broker schema must exist and be usable by the app user.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Copy link
Contributor

@adamdickmeiss adamdickmeiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JanisSaldabols JanisSaldabols merged commit 0f9f410 into main Feb 11, 2026
3 checks passed
@JanisSaldabols JanisSaldabols deleted the CROSSLINK-203 branch February 11, 2026 13:46
JanisSaldabols added a commit that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants